Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add more debugging output / destroy api when reconnect fails #163

Merged
merged 10 commits into from
Feb 19, 2020

Conversation

David-Development
Copy link
Member

@David-Development David-Development commented Jan 21, 2020

Closes #162

@David-Development
Copy link
Member Author

@stefan-niedermann @desperateCoder did you guys manage to test the changes proposed in the MR?

@stefan-niedermann
Copy link
Member

Not yet, sry

David-Development and others added 9 commits February 18, 2020 16:46
… logging

Signed-off-by: David Luhmer <david-dev@live.de>
Signed-off-by: David Luhmer <david-dev@live.de>
Signed-off-by: David Luhmer <david-dev@live.de>
Signed-off-by: David Luhmer <david-dev@live.de>
Signed-off-by: David Luhmer <david-dev@live.de>
… logging

Signed-off-by: David Luhmer <david-dev@live.de>
@nextcloud-android-bot
Copy link
Collaborator

Codacy

29

Lint

TypemasterPR
Warnings22
Errors00

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings2
Correctness Warnings15
Internationalization Warnings3
Multithreaded correctness Warnings3
Performance Warnings5
Security Warnings5
Dodgy code Warnings9
Total42

SpotBugs (master)

Warning TypeNumber
Bad practice Warnings2
Correctness Warnings12
Internationalization Warnings3
Multithreaded correctness Warnings3
Performance Warnings5
Security Warnings5
Dodgy code Warnings9
Total39

@David-Development David-Development changed the title WIP: add more debugging output / destroy api when reconnect fails add more debugging output / destroy api when reconnect fails Feb 19, 2020
@tobiasKaminsky tobiasKaminsky merged commit 9c93c87 into master Feb 19, 2020
@delete-merged-branch delete-merged-branch bot deleted the fix-api-not-responding branch February 19, 2020 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NextcloudApiNotRespondingException
5 participants